Skip to content

Conversation

@creesch
Copy link
Owner

@creesch creesch commented Dec 16, 2024

Alright, actually got it working. Fixes #1

Replaces NanoHTTPD in favor of javalin giving a cleaner and more modern implementation.

@creesch creesch changed the title Rewrite web interface using javalin Rewrite web interface using javalin. Fixes #1 Dec 16, 2024
@creesch creesch marked this pull request as ready for review December 16, 2024 21:19
@creesch creesch changed the title Rewrite web interface using javalin. Fixes #1 Rewrite web interface replacing NanoHTTPD in favor of Javalin. Fixes #1 Dec 16, 2024
@creesch
Copy link
Owner Author

creesch commented Dec 16, 2024

Aaaand build is broken again... not sure what is going on with dependencies here.

@danthedaniel
Copy link
Collaborator

Does this obsolete #12?

@creesch
Copy link
Owner Author

creesch commented Dec 16, 2024

Does this obsolete #12?

Once I git it to actually work it will. So far I am having trouble building properly with the dependencies needed. I could have sworn it build fine with this build.gradle but after merging in the main branch (and messing that up somehow as well) it broke on me again even though I don't think anything meaningfull changed there.

@creesch
Copy link
Owner Author

creesch commented Dec 16, 2024

Alright... threw away .gradle entirely in my home directory and now it builds again locally. Let's double check if artifacts also contain proper builds.

Edit:

And of course the artifacts created by building on github are broken... I don't get what is going on here.

@creesch
Copy link
Owner Author

creesch commented Dec 17, 2024

Alright, took some work but finally got it working properly using the shadow plugin.

@creesch creesch merged commit cd60724 into main Dec 17, 2024
2 checks passed
@creesch creesch deleted the rewrite-web-interface-using-Javalin branch May 4, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Switch out NanoHTTPD and websockets in favor of Javalin

3 participants